Skip to content

Remove default xml parser, allow users to use their own libraries.#16

Closed
thiccaxe wants to merge 2 commits intomasterfrom
remove-xml-parse
Closed

Remove default xml parser, allow users to use their own libraries.#16
thiccaxe wants to merge 2 commits intomasterfrom
remove-xml-parse

Conversation

@thiccaxe
Copy link
Copy Markdown
Member

bumps version to 1.0.0, as output of API is now by default an XML string.

@thiccaxe
Copy link
Copy Markdown
Member Author

thiccaxe commented Jul 18, 2023

also bumps SOAP to the latest 1.0.0, which has up-to-date packages which had vulnerabilities.

closes #15

Copy link
Copy Markdown
Member

@CoolBro4811 CoolBro4811 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good?

@kaijchang kaijchang self-requested a review May 17, 2024 17:32
Copy link
Copy Markdown
Member

@kaijchang kaijchang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't really see why this is helpful.

@kaijchang kaijchang closed this May 17, 2024
@kaijchang kaijchang deleted the remove-xml-parse branch May 17, 2024 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants